-
-
Notifications
You must be signed in to change notification settings - Fork 682
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Ports the RBMK from NSV (for real this time) #10364
Ports the RBMK from NSV (for real this time) #10364
Conversation
This pull request has conflicts, please resolve those before we can evaluate the pull request. |
To do: On initialization, make new ID for the RBMK and have computers auto-locate and set their own IDs to it Create and make the crate use the RBMK machine board instead of the machine + add the ability to move the RBMK when inactive
now onto the rest of the TODO list...
credit to: https://freesound.org/people/guitarguy1985/sounds/54048/ license: CC0 1.0 Deed
If it is hidden in some corner of the game, it will be unmaintained because no one will ever go to that corner and know of its existence. I would say we SHOULD allow the SM to be replaced with other engines if the situation desires it. Because then people will actually care for it and its functionality further into the feature. With how hidden Derelict Stations are, it is no wonder how unmaintained it is. Not because PRs are being shut down, but because no one knows of its existence to make the PRs to begin with. |
A new mapping standard won't be set for stations to have their SM replaced with RBMK. We will suffer from having to maintain multiple engines, and the RBMK is not an appropriate option for a full SM replacement. |
I really love the RBMK, but I ultimately agree with Bacon. I don't want to replace the supermatter crystals with it, and I don't believe this should be merged unless it is replacing them. |
Please dont kill it, from initial thought it was supposed to replace the defunct antimatter reactor, which it does a pretty good job, never meant to replace the SM in any way and engineering desperately needs new content to well.. play around gameplay loop currently is really vague It's a replacement incase the SM explodes, not a full replacement for the SM No clue where Ambic got the full replacement for the SM idea, it was always supposed and designed to be a secondary engine, not a main one |
Almost a year in development and now the close keys are coming in? Jesus Christ. A year of Xeons life wasted. |
I dont mind to be fair. I code as a hobby, something I do for fun. Not for a living. |
But yes, the idea from the start was as a replacement for the now gone antimatter reactor. As of current, if the SM explodes, the station only has solars as a power source unless they want to take a rather long time setting up a black hole (good luck with no power) or a tesla (Again, good luck without power). Or engineering could just set up a second SM in the form of a shard, but I'm unsure if they'd put that much effort in before the shuttle is called. |
I am honestly fine with this, but I suppose we might need a kind of mapping tool to replace (or some ingame stuff to make a dedicated area quickly). Constructing a new engine for a whole round was tedious. |
Entirely fair, if wished, i am able to add one. |
not right now. we are in a mapping disaster rn. |
This comment was marked as off-topic.
This comment was marked as off-topic.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Eh whatever, this can replace the antimatter generator.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Replace arbitrary names like R or N. It tells no context.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Code looks alright as long as change requests are applied.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Would be a final.
Co-authored-by: EvilDragonfiend <[email protected]>
Co-authored-by: EvilDragonfiend <[email protected]>
Co-authored-by: EvilDragonfiend <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
okay, we'll see how this goes.
YES YES YES YES |
About The Pull Request
Adds in the RBMK nuclear reactor from NSV. (this time hopefully without pull breaking map changes)
All things are able to be ordered from cargo.
Wiki can be found here: https://wiki.beestation13.com/view/Guide_to_the_Nuclear_Reactor
(I accidentally created the wiki on the main wiki instead of a sandbox, I'm sorry)
RBMK Hum noise sourced from here, under CC BY 4.0 and edited by me.
Why It's Good For The Game
More content for engineering!
Testing Photographs and Procedure
Screenshots&Videos
Changelog
🆑 XeonMations, Aramix, DungeonVR
add: Added the RBMK reactor, fuel rods and engine sealant, all orderable from cargo!
/:cl: